Skip to content

feat: Added realistic template for sd-tag#2889

Merged
balco0110 merged 14 commits intomainfrom
feat/realistic-sd-tag-template
Apr 24, 2026
Merged

feat: Added realistic template for sd-tag#2889
balco0110 merged 14 commits intomainfrom
feat/realistic-sd-tag-template

Conversation

@balco0110
Copy link
Copy Markdown
Contributor

@balco0110 balco0110 commented Apr 7, 2026

Description:

Closes: #2134

Definition of Reviewable:

  • E2E tests (features, a11y, bug fixes) are created/updated
  • Stories (features, a11y) are created/updated
  • relevant tickets are linked

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 7, 2026

🦋 Changeset detected

Latest commit: acd7419

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@solid-design-system/docs Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@balco0110 balco0110 moved this from 📋 Backlog to 👀 In review in Solid Design System Project Board Apr 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 7, 2026

🚀 Storybook has been deployed for branch feat_realistic-sd-tag-template

Copy link
Copy Markdown
Contributor

@MartaPintoTeixeira MartaPintoTeixeira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This things are not as in figma:

  • Remove "Risk" in each tag of the section "Filter Tag Group with Risk"
  • "Moderate" instead of "Moderated" in section "Filter Tag Group with Risk"
  • In this section "Removable Filter Tag Group" the underline of "Remove..." is starting before the R (it has one space more then it should)
  • In this section "Removable Filter Tag Group" in figma the space between "ETFs" tag and "Remove filters" is bigger then in code. in figma "Remove filters has a left padding of 16.
  • in this section "Tag Group with Links" tags are suposed to hug horizontally the content as in figma. they should not have all the same horizontal size.

Assign me again once solved. I will remove my assignment

@MartaPintoTeixeira MartaPintoTeixeira removed their assignment Apr 7, 2026
…xed the wrong underline start and added margin between tags and link and fixed horizontal size
@auroraVasconcelos
Copy link
Copy Markdown
Contributor

The sd-tag overview page references the template "Tag". I think for consistency, it should reference the different stories "Filter Tag Group" "Filter Tag Group with Morningstart Rating" etc, right?

Hey @martascorreia, when linking a template, use the component’s name if it’s the main component of that template (like in this case). Only use the story name when the template includes multiple components and the one you’re linking is not the main one.

You can find more details about documenting in Storybook here: https://solid-design-system.fe.union-investment.de/docs/?path=/docs/packages-docs-contributing--docs

Comment thread packages/docs/src/stories/templates/tag.stories.ts Outdated
Comment thread packages/docs/src/stories/templates/tag.stories.ts Outdated
Comment thread packages/docs/src/stories/templates/tag.stories.ts Outdated
Comment thread packages/docs/src/stories/templates/tag.stories.ts Outdated
@smfonseca smfonseca removed their assignment Apr 21, 2026
@smfonseca smfonseca removed their assignment Apr 24, 2026
@balco0110 balco0110 merged commit fcefc52 into main Apr 24, 2026
19 checks passed
@balco0110 balco0110 deleted the feat/realistic-sd-tag-template branch April 24, 2026 14:58
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Solid Design System Project Board Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

feat[dev]: ✨ Realistic sd-tag template

7 participants